Skip to content

Conversation

@gproly
Copy link
Contributor

@gproly gproly commented Feb 10, 2026

Description:
feat(end event): support Mustache in external URL for element destination

  • Resolve Mustache expressions in end event "External URL" when the token reaches the end event, using the same context as scripts/screens (APP_URL, _request, _user, process variables).
  • Add getElementDestinationMustacheContext() to build context via DataManager with fallback; normalize to plain array for Mustache.
  • Add resolveElementDestinationUrl() to decode HTML entities and render URL template with MustacheExpressionEvaluator. FEEL is not supported.
  • Apply resolution only for externalURL type; conditional redirect URLs also go through Mustache when destination is external URL.
  • Harden getElementDestinationAttribute(): ensure conditionalRedirectProp and elementDestinationProp are never null (json_decode ?? [], pass ?? []).
  • Modeler: allow URL validation when string contains {{ (Mustache); update helper and error copy to document _request.id, _user.id, process vars.

ci:deploy
ci:modeler:feature/FOUR-29250
Related tickets:
https://processmaker.atlassian.net/browse/FOUR-29250

Description:
feat(end event): support Mustache in external URL for element destination

- Resolve Mustache expressions in end event "External URL" when the token
  reaches the end event, using the same context as scripts/screens
  (APP_URL, _request, _user, process variables).
- Add getElementDestinationMustacheContext() to build context via DataManager
  with fallback; normalize to plain array for Mustache.
- Add resolveElementDestinationUrl() to decode HTML entities and render
  URL template with MustacheExpressionEvaluator. FEEL is not supported.
- Apply resolution only for externalURL type; conditional redirect URLs
  also go through Mustache when destination is external URL.
- Harden getElementDestinationAttribute(): ensure conditionalRedirectProp
  and elementDestinationProp are never null (json_decode ?? [], pass ?? []).
- Modeler: allow URL validation when string contains {{ (Mustache); update
  helper and error copy to document _request.id, _user.id, process vars.
Related tickets:
https://processmaker.atlassian.net/browse/FOUR-29250
@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-4a306d1d3c.engk8s.processmaker.net

$context['APP_URL'] = config('app.url');

// Normalize to plain arrays/scalars so Mustache resolves all keys (common PHP idiom)
$normalized = json_decode(json_encode($context), true);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add JSON_THROW_ON_ERROR to surface encoding failures

Suggested change
$normalized = json_decode(json_encode($context), true);
$json = json_encode($context, JSON_THROW_ON_ERROR);
$normalized = json_decode($json, true, 512, JSON_THROW_ON_ERROR);

gproly and others added 2 commits February 11, 2026 15:56
Co-authored-by: Miguel Angel <miguelangellodev@gmail.com>
Co-authored-by: Miguel Angel <miguelangellodev@gmail.com>
@processmaker-sonarqube
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants